Skip to content

Conversation

@craigtaverner
Copy link
Contributor

This reverts commit 4d3e27f.

Yesterday we merged two performance updates to ES|QL Spec tests that had been reverted two months back after noticing some increased test flakiness. However, we immediately saw increased failures due to one of them, so we reverted it. Having done that we continue to see other failures, that may or may not be related to the remaining performance update. This PR reverts that too.

…resilience improvement to EsqlSpecTestCase (elastic#136781)"

This reverts commit 4d3e27f.
@craigtaverner craigtaverner added >test Issues or PRs that are addressing/adding tests Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Analytics/ES|QL AKA ESQL labels Nov 4, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@elasticsearchmachine elasticsearchmachine merged commit 8e4ad76 into elastic:main Nov 4, 2025
34 checks passed
@craigtaverner craigtaverner deleted the revert_performace_updates branch November 4, 2025 18:11
craigtaverner added a commit to craigtaverner/elasticsearch that referenced this pull request Nov 10, 2025
… resilience improvement to EsqlSpecTestCase (elastic#136781)" (elastic#137589)

This reverts commit 8e4ad76.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants